Skip to content

Better CQL handling#474

Merged
kalxas merged 1 commit intogeopython:masterfrom
tomkralidis:cql-sru-query-handling
Oct 11, 2016
Merged

Better CQL handling#474
kalxas merged 1 commit intogeopython:masterfrom
tomkralidis:cql-sru-query-handling

Conversation

@tomkralidis
Copy link
Copy Markdown
Member

@tomkralidis tomkralidis commented Oct 9, 2016

Overview

This PR adds support for more robust CQL and SRU query input handling:

  • transform CQL queries into OGC Filter queries by implementing a CQL parser
    • CQL parser initially supports basic conditions and AND / OR logical operators
  • suppress GetRecords database errors in ogc:ExceptionReport XML

Related Issue / Discussion

NA

Additional Information

This also applies to SRU query handling given SRU queries are transformed into CQL queries.

Contributions and Licensing

(as per https://github.com/geopython/pycsw/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to pycsw. I confirm that my contributions to pycsw will be compatible with the pycsw license guidelines at the time of contribution.
  • I have already previously agreed to the pycsw Contributions and Licensing Guidelines

@tomkralidis tomkralidis force-pushed the cql-sru-query-handling branch 2 times, most recently from 3e215fa to 8f3e4cf Compare October 9, 2016 14:54
@kalxas kalxas changed the title [WIP] DO NOT MERGE: better CQL handling Better CQL handling Oct 9, 2016
@kalxas kalxas self-assigned this Oct 9, 2016
@kalxas
Copy link
Copy Markdown
Member

kalxas commented Oct 9, 2016

Looks like we have a working solution here.
@tomkralidis @ricardogsilva what do you think?

@tomkralidis tomkralidis force-pushed the cql-sru-query-handling branch from 8f3e4cf to 3b1ffd0 Compare October 9, 2016 22:25
@tomkralidis tomkralidis force-pushed the cql-sru-query-handling branch from ccd7537 to d83df94 Compare October 11, 2016 12:01
@kalxas
Copy link
Copy Markdown
Member

kalxas commented Oct 11, 2016

I am +1 to merge this PR

@kalxas kalxas merged commit 2565ab3 into geopython:master Oct 11, 2016
@tomkralidis tomkralidis deleted the cql-sru-query-handling branch November 3, 2016 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants